-
Notifications
You must be signed in to change notification settings - Fork 54
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
change zoom with (ctrl + mouse wheel) #236
base: develop
Are you sure you want to change the base?
change zoom with (ctrl + mouse wheel) #236
Conversation
Hey @Eng-Elias! Sorry for slow reply. This looks dope, and I like the idea of supporting this. I hope we can move it into the viewer though. It's unclear from the repo (there's no CONTRIBUTING guide like we should probably have), but your change was made to the demo application and not the viewer itself. I think you could make a somewhat similar change to the Line 21 in ea558f2
Maybe with Line 253 in ea558f2
How to percolate that change is unfortunately tricky. You'll need to:
|
Hi @jjti , sorry for the delay to respond but I moved to Dubai this month and I was very busy. |
…conflicts # Conflicts: # src/SeqViz.tsx
Hi @jjti , I embedded mouse wheel zoom into viewer instead of demo, please review the changes and let me know if you have any note. |
make changing zoom easier by using (ctrl + mouse wheel)